Skip to content

Use Failsafe to retry instead iceberg's Tasks in iceberg connector#15758

Merged
ebyhr merged 1 commit intotrinodb:masterfrom
krvikash:use-FailSafe-in-iceberg
Jan 19, 2023
Merged

Use Failsafe to retry instead iceberg's Tasks in iceberg connector#15758
ebyhr merged 1 commit intotrinodb:masterfrom
krvikash:use-FailSafe-in-iceberg

Conversation

@krvikash
Copy link
Copy Markdown
Contributor

Description

Inspired from #14869 (comment)

Release notes

(X) This is not user-visible or docs only and no release notes are required.

@cla-bot cla-bot Bot added the cla-signed label Jan 18, 2023
@krvikash krvikash added the no-release-notes This pull request does not require release notes entry label Jan 18, 2023
Copy link
Copy Markdown
Member

@alexjo2144 alexjo2144 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@krvikash krvikash force-pushed the use-FailSafe-in-iceberg branch from d9096c5 to 7f8d928 Compare January 18, 2023 18:24
@krvikash
Copy link
Copy Markdown
Contributor Author

Rebased with upstream master.

@krvikash krvikash force-pushed the use-FailSafe-in-iceberg branch from 7f8d928 to d5e93aa Compare January 19, 2023 03:16
@krvikash
Copy link
Copy Markdown
Contributor Author

Addressed comments.

@ebyhr ebyhr merged commit 7facf0e into trinodb:master Jan 19, 2023
@krvikash krvikash deleted the use-FailSafe-in-iceberg branch January 19, 2023 07:00
@github-actions github-actions Bot added this to the 406 milestone Jan 19, 2023
@findepi
Copy link
Copy Markdown
Member

findepi commented Jan 19, 2023

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed no-release-notes This pull request does not require release notes entry

Development

Successfully merging this pull request may close these issues.

4 participants